[multiple] Add bootc CP job and wire EDPM image outputs#3952
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider FAILURE in 21m 21s |
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider FAILURE in 27m 19s |
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider TIMED_OUT in 31m 24s |
|
recheck |
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider TIMED_OUT in 31m 24s |
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider TIMED_OUT in 31m 24s |
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider TIMED_OUT in 31m 24s |
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider TIMED_OUT in 31m 26s |
rabi
force-pushed
the
bootc
branch
2 times, most recently
from
e2eb0c4 to
e786602
Compare
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider FAILURE in 21m 07s |
rabi
force-pushed
the
bootc
branch
2 times, most recently
from
8fc6ca8 to
7a0b6f4
Compare
rabi
force-pushed
the
bootc
branch
4 times, most recently
from
4287f7c to
9213830
Compare
|
Zuul encountered a syntax error while parsing its The project template "podified-multinode-edpm-baremetal-bootc- The problem appears in the "openstack-k8s-operators/edpm-ansible" project stanza: project: in "openstack-k8s-operators/edpm-ansible/zuul.d/projects.yaml@main", line 2 |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 39m 32s |
rabi
force-pushed
the
bootc
branch
2 times, most recently
from
bc0dcda to
0180318
Compare
rabi
force-pushed
the
bootc
branch
3 times, most recently
from
f39edc3 to
39bb2ae
Compare
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 4h 20m 06s |
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider FAILURE in 20m 21s |
rabi
changed the title
evallesp
left a comment
evallesp
left a comment
There was a problem hiding this comment.
I've done a quick review. Please answer the threads and I'll continue.
Question: is it possible to split the code here in more than one commit and/or MRs? I see this is grouping a single functionality, but makes the review process much more slower.
| > {{ cifmw_edpm_build_images_basedir }}/logs/edpm_images/edpm_bootc_qcow2_build.log | ||
| 2> {{ cifmw_edpm_build_images_basedir }}/logs/edpm_images/edpm_bootc_qcow2_build_err.log | ||
|
|
||
| - name: Rename generated bootc qcow2 image |
There was a problem hiding this comment.
(blocking) suggestion: Let's move to use copy module.
There was a problem hiding this comment.
Not sure why? This is a large image and we should not duplicate it.
There was a problem hiding this comment.
Definetely sorry because my comment was too short.
I mean about the pipelines to the normal exit and the error exit.
There was a problem hiding this comment.
Oh you mean use copy for the logs, not the qcow2. register + copy would work but adds 4 tasks and buffers all build output in memory. Keeping > / 2>: we already get separate stdout/stderr log files streamed to disk, Ansible still fails on non-zero exit, and the error log is written even when the build fails. I did not see the pattern of register+copy elsewhere in the repo for logs.
Hey @rabi, thanks for the PR! Before moving on to the analysis and code review, I definitely want to highlight @evallesp's comments about splitting this implementation into multiple PRs.
Of course, this is just a suggestion, not necessarily what I expect you to do. |
Single PR is intentional: bootc build/push, content-provider wiring, and Zuul/deploy changes belong together and are only fully tested by the new bootc pipeline jobs. Splitting would not add review or CI value, because intermediate PRs would land with incomplete, untested functionality. |
rabi
force-pushed
the
bootc
branch
2 times, most recently
from
423f6c0 to
d6fb903
Compare
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider TIMED_OUT in 31m 30s |
|
Build failed (check pipeline). Post ❌ openstack-k8s-operators-content-provider TIMED_OUT in 31m 25s |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 40m 01s |
|
rechck RETRY_LIMIT |
Add a bootc-specific content provider job and wire the bootc baremetal pipeline to it. Also change cifmw-content-provider-build-imagesto build and push both bootc and non-bootc images. Change-Id: I4cd8035334328459f1ec298133d1f8d0ec0e0a38 Signed-off-by: rabi <ramishra@redhat.com>
3 participants
Add a bootc-specific content provider job alongside the existing UEFI content provider flow, and wire baremetal jobs to consume the returned EDPM OS container image outputs. Also gate EDPM image publishing on edpm-image-builder changes, with ci-framework overrides to validate the wiring here.